[문서] C12 충돌 해결#1857
Conversation
|
|
This comment was marked as low quality.
This comment was marked as low quality.
Sorry, something went wrong.
| ======= | ||
| There will be no error because of "excessive" arguments. But of course in the result only the first two will be counted, so the result in the code above is `3`. | ||
| >>>>>>> upstream/master | ||
| 함수를 정의할 땐 인수를 두 개만 받도록 하고, 실제 함수를 호출할 땐 이보다 더 많은 '여분의' 인수를 전달했지만, 에러가 발생하지 않았습니다. 다만 결과를 계산할 때는 처음 두 개의 인수만 사용하므로 위 코드의 결과는 `3`이 됩니다. |
There was a problem hiding this comment.
뒤 문장 번역해 주신 부분은 잘 해주신 거 같은데 앞 문장이 '여분의' 이 부분이 잘드러나고 간략하면 좋을 거 같은데 아래와 같은 표현은 어떠신가요?
'여분의' 인수가 있어도 에러는 발생하지 않습니다. 다만 결과를 계산할 때는 처음 두 개의 인수만 사용하므로 위 코드의 결과는 '3'이 됩니다.
There was a problem hiding this comment.
저도 처음에는 그렇게 생각했는데 이전에 번역하신 분이 앞의 내용까지 고려해서 지금처럼 번역을 해주신 것 같더라구요.
앞서 번역을 해주신 분의 의도가 저는 해석하는 데에 있어서 더 좋다고 판단되어서 이전의 번역을 최대한 건들지 않고 뒷 문장만 번역을 추가하려고 해서 지금과 같은 번역이 완성 되었습니다.
앞에 실제 예시로 함수 SUM에 인수를 2개 넣도록 선언하고 5개의 인수를 넣는 예시가 있는데 이러한 예시를 고려한 한글 번역이 영어 번역보다 더 적절하다고 생각해서 이렇게 번역을 유지했습니다.
There was a problem hiding this comment.
처음 읽는 사람 관점에서 코드의 맥락도 생각하면 기존 번역이 자연스러운 거 같네요!
그럼 리뷰 완료하겠습니다
DevJunz
force-pushed
the
fix/devjunz-c12-conflict
branch
from
ff34d6d to
e8c94f4
Compare
|
@Violet-Bora-Lee 커밋메세지 수정 했습니다 |
jhroom
left a comment
jhroom
left a comment
There was a problem hiding this comment.
안녕하세요 전체적으로 다시 리뷰를 진행해 보았습니다. 다른 곳은 문제 되는 곳이 안보이고
한 부분에 대해 리뷰 남깁니다.
| let arrCopy = [...arr]; // spread the array into a list of parameters | ||
| // then put the result into a new array | ||
| let arrCopy = [...arr]; // 배열을 펼쳐서 각 요소를 분리후, 매개변수 목록으로 만든 다음에 | ||
| // 매개변수 목록을 새로운 배열에 할당함 |
There was a problem hiding this comment.
맞춤법상 '분리 후'가 맞는 표현으로 수정해야 할 것 같습니다.
그리고 분리가 펼친다는 동작과 이어지는 동사적 흐름으로 '분리한 후'가 매끄럽다는 부분도 있는 것 같아 의견을 여쭙고 싶습니다.
저는 spread가 행위의 결과로서 동사적 표현이 조금 더 가깝지 않나 생각합니다.
| let objCopy = { ...obj }; // spread the object into a list of parameters | ||
| // then return the result in a new object | ||
| let objCopy = { ...obj }; // 객체를 펼쳐서 각 요소를 분리후, 매개변수 목록으로 만든 다음에 | ||
| // 매개변수 목록을 새로운 객체에 할당함 |
5 participants
요약
충돌 해결 파일 리스트
Pull Request 체크리스트
TODO